Skip to content

Variables example #194

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: camel-quarkus-main
Choose a base branch
from
Open

Variables example #194

wants to merge 1 commit into from

Conversation

@JiriOndrusek
Copy link
Contributor

Example of Variables (new feature in Camel 4.4) - apache/camel-quarkus#5620

Can be merged once Camel 4.4 is part of the CQ main branch

This was referenced Jan 17, 2024
Merged
Open
@aldettinger
Copy link
Contributor

@JiriOndrusek This pr might need to be rebased after the adoption of quarkus platform release 3.9.1

@JiriOndrusek
Copy link
Contributor Author

@JiriOndrusek This pr might need to be rebased after the adoption of quarkus platform release 3.9.1

Thanks!
I noticed this PR yesterday, and I plan to rebase it.

@JiriOndrusek JiriOndrusek marked this pull request as ready for review March 28, 2024 07:51
@JiriOndrusek
Copy link
Contributor Author

Failed test is not related to this PR. It is ChuckNorrisTest.testTimerLogMain

aldettinger
aldettinger reviewed Mar 28, 2024
View reviewed changes
variables/README.adoc

[source,shell]
----
$ mvn clean package -Pnative
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's advised to use -Dnative since a few quarkus versions.

aldettinger
aldettinger reviewed Mar 28, 2024
View reviewed changes
variables/pom.xml

<artifactId>camel-quarkus-examples-variables</artifactId>
<groupId>org.apache.camel.quarkus.examples</groupId>
<version>3.7.0-SNAPSHOT</version>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably we need to update the version here ?

aldettinger
aldettinger reviewed Mar 28, 2024
View reviewed changes
variables/pom.xml
<type>pom</type>
<scope>import</scope>
</dependency>
<dependency>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we need to import camel-quarkus-bom-test please ?

aldettinger
aldettinger reviewed Mar 28, 2024
View reviewed changes
variables/pom.xml
<artifactId>rest-assured</artifactId>
<scope>test</scope>
</dependency>
<dependency>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we really need the process-executor-support ?

@aldettinger aldettinger force-pushed the camel-quarkus-main branch 4 times, most recently from 4937107 to 017fdf1 Compare June 26, 2024 15:03
@zbendhiba zbendhiba force-pushed the camel-quarkus-main branch 2 times, most recently from 773f362 to 2931bea Compare December 12, 2024 14:08
@ppalaga ppalaga force-pushed the camel-quarkus-main branch from d29788f to f2392be Compare October 3, 2025 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aldettinger aldettinger aldettinger left review comments

@oscerd oscerd oscerd approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@JiriOndrusek @aldettinger @oscerd